Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: unenroll request when a refund is processed #41

Merged
merged 3 commits into from
Sep 27, 2023

Conversation

julianramirez2
Copy link
Contributor

Description

This feature enables the operator, when processing a refund for a course product, to automatically generate an 'un-enroll' request within the Open edX platform, thus streamlining and expediting the process. Furthermore, the operator will have access to corresponding logs indicating whether the process was successful or not, along with links to the new request.

Testing instructions

To test this, you should navigate to an order and create it either manually or using the existing PayPal process to generate a new order. Once the order status is 'processing,' click on the 'refund' button, select the amount, and click 'accept.' Subsequently, the entire process will appear in the order notes, and the associated Enrollment Request ID will be updated.

This is how the refund looks like:
image

Additional information

This process only covers the happy path, meaning that more complicated error cases, such as edge cases, may cause the process to fail or trigger a fatal WordPress error.

Checklist for Merge

  • Tested in a remote environment
  • Updated documentation
  • Rebased master/main
  • Squashed commits

@julianramirez2
Copy link
Contributor Author

@MaferMazu @andres-arango-aptitude can you check this, please?

Copy link
Contributor

@MaferMazu MaferMazu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@julianramirez2 julianramirez2 merged commit 6021b6c into main Sep 27, 2023
3 checks passed
@felipemontoya felipemontoya deleted the unenroll-handler branch March 21, 2024 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants